Skip to content

Conversation

slawekptak
Copy link
Contributor

The kernel function pointer, used to extract the kernel arguments, needs to point to the user provided function, or a function copy, if the copy/move is needed.

The kernel function pointer, used to extract the kernel arguments,
needs to point to the user provided function, or a function copy,
if the copy/move is needed.
Copy link
Contributor

@mmichel11 mmichel11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I verified this fix resolves observed issues of kernel argument capture with graph recording

Copy link
Contributor

@reble reble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mmichel11 We should also add your regression test, can do as a follow-up PR.

Copy link
Contributor

@aelovikov-intel aelovikov-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test is needed in this PR.

@slawekptak
Copy link
Contributor Author

@intel/llvm-gatekeepers please consider merging

@steffenlarsen steffenlarsen merged commit febf259 into intel:sycl Oct 21, 2025
61 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants